Skip to content

Conversation

@lawn123
Copy link
Contributor

@lawn123 lawn123 commented Jun 13, 2024

Add a new archecture LoongArch64 for mono.This PR is about loongarch support in some files.

…ngArch64

Change-Id: I52c8c227fdeeee840cad14496e24e93a04521143
@ghost ghost added the area-Build-mono label Jun 13, 2024
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Jun 13, 2024
@lawn123
Copy link
Contributor Author

lawn123 commented Jun 13, 2024

@shushanhf

set(CMAKE_SYSTEM_PROCESSOR "x86")
elseif(CMAKE_SYSTEM_PROCESSOR STREQUAL "aarch64" OR CMAKE_SYSTEM_PROCESSOR STREQUAL "ARM64")
set(CMAKE_SYSTEM_PROCESSOR "arm64")
elseif(CMAKE_SYSTEM_PROCESSOR STREQUAL "loongarch64" OR CMAKE_SYSTEM_PROCESSOR STREQUAL "LOONGARCH64")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also add

runtime/src/mono/mono.proj

Lines 457 to 461 in 4f9cd4d

<!-- riscv64 Linux cross build options -->
<ItemGroup Condition="'$(MonoCrossDir)' != '' and '$(TargetArchitecture)' == 'riscv64'">
<_MonoCMakeArgs Include="-DCMAKE_TOOLCHAIN_FILE=$(CrossToolchainFile)" />
<_MonoBuildEnv Include="TARGET_BUILD_ARCH=riscv64" />
</ItemGroup>
to enable LA64 mono cross-build.

@am11 am11 requested a review from akoeplinger June 13, 2024 09:18
@lawn123
Copy link
Contributor Author

lawn123 commented Jun 13, 2024

@am11 Thank you very much. I have added the code you suggested to this PR.

@dotnet-policy-service
Copy link
Contributor

Draft Pull Request was automatically closed for 30 days of inactivity. Please let us know if you'd like to reopen it.

@github-actions github-actions bot locked and limited conversation to collaborators Dec 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

arch-loongarch64 area-Build-mono community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants